Skip to content

feat(alarms): show Apple location and car reminders - #3306

Closed
chanha0406 wants to merge 2 commits into
nextcloud:mainfrom
chanha0406:feat/apple-proximity-alarms
Closed

feat(alarms): show Apple location and car reminders#3306
chanha0406 wants to merge 2 commits into
nextcloud:mainfrom
chanha0406:feat/apple-proximity-alarms

Conversation

@chanha0406

@chanha0406 chanha0406 commented Oct 20, 2026

Copy link
Copy Markdown

Summary

  • iOS Reminders stores location and car alerts as VALARM with X-APPLE-PROXIMITY (ARRIVE / DEPART / CONNECT / DISCONNECT) and a dummy TRIGGER of 19760401T005545Z.
  • Tasks currently treats that trigger as a wall-clock time, so the sidebar shows 1976 (often read as 1970).
  • Parse the Apple properties, show “when getting in/out of the car” or “when arriving at/leaving {place}”, and disable time-edit so a save does not rewrite the proximity VALARM.

Display and delete only. This does not add a place picker, and Nextcloud still cannot fire geofences.

No existing issue or PR for X-APPLE-PROXIMITY turned up on nextcloud/tasks.

Test plan

  • Unit tests in tests/javascript/unit/utils/appleProximity.spec.js
  • Open an iOS Reminders task with “when getting in the car” and confirm the reminder label is not 1976
  • Open arrive/leave location reminders and confirm the place title from X-TITLE is shown
  • Confirm “Edit time” is hidden for those reminders and that deleting still works
  • Confirm a normal timed VALARM is unchanged

Made with Cursor

chanha0406 and others added 2 commits October 20, 2026 18:33
@chanha0406 @cursoragent
feat(alarms): show Apple location and car reminders
36362a8 
iOS writes X-APPLE-PROXIMITY VALARMs with a dummy 1976 trigger.
Display those as car/arrive/leave text instead of a calendar date,
and do not offer a time edit that would rewrite the VALARM.

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: ChanHa Ryu <chanha0406@gmail.com>
@chanha0406 @cursoragent
chore: keep 0.18.1 for the upstream pull request
c848ad1 
Version bumps belong in the release, not this display fix.

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: ChanHa Ryu <chanha0406@gmail.com>
@chanha0406
chanha0406 force-pushed the feat/apple-proximity-alarms branch from 4844a99 to c848ad1 Compare October 20, 2026 09:33
@chanha0406 chanha0406 closed this Oct 20, 2026
@chanha0406
chanha0406 deleted the feat/apple-proximity-alarms branch October 20, 2026 09:53
@chanha0406
chanha0406 restored the feat/apple-proximity-alarms branch October 20, 2026 12:45
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

feedback-requested

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@chanha0406